-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix warning on conversion of anon func with implicit parameters #59522
Conversation
ab718e7
to
8bd26e5
Compare
8bd26e5
to
cb33d99
Compare
src/Compilers/CSharp/Test/Semantic/Semantics/NullableReferenceTypesTests.cs
Show resolved
Hide resolved
@@ -1357,7 +1356,6 @@ internal sealed class PlainUnboundLambdaState : UnboundLambdaState | |||
_isStatic = isStatic; | |||
} | |||
|
|||
public override bool HasNames { get { return !_parameterNames.IsDefault; } } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so this member was just totally redundant in practice?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, they have the same meaning. We had HasNames
, HasSignature
and I was adding yet another one...
Fixes #59336